Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

seasonizer; Frage/Problem bzgl. declaration compiler error #8

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

FelixMei
Copy link

Hey Nils,

vielleicht weißt Du was hinter der Fehlermeldung steckt, die irgendwie im Zusammenhang zu den guard-statements steht?

Danke und Gruß
Felix

@nilsvu
Copy link
Member

nilsvu commented May 22, 2016

Der Code steht einfach in der Klasse, das macht keinen Sinn, denn da wird ja nie Code ausgeführt. In Klassen muss Code immer in einer Methode oder dergleichen stehen, die dann irgendwann ausgeführt werden kann. In den TODOs steht, welche Methoden implementiert werden müssen. Überleg dann mal, was da passiert, also versuch nachzuverfolgen, wie die Accessory Daten durch die App "fließen".

@FelixMei
Copy link
Author

Hallo Nils, danke für den Tipp! Also es läuft jetzt zwar, und den Flow habe ich mir auch soweit möglich klar gemacht, aber trotzdem tut die Anzeige immer noch nichts und in der Accessory List werden auch die entsprechenden Items nicht aufgelistet...Bitte um weiteren Support :)

@FelixMei
Copy link
Author

Hey Nils,

danke für dein Feedback! ….Hast Du meine zwischenzeitlichen Verbesserungen angeschaut (das hab ich in einer Pull-Request aktualisiert..hoffe, da ist nichts schief gelaufen?) und könntest mir dazu noch ein Feedback/Support geben (Ich war am Montag deswegen nach der Vorlesung bei Dir :)…) ?

Zur aktuellen Aufgabe habe ich auch einen Pull-Request; mir fehlt noch ein bisschen der Überblick über die gesamten erforderlichen Komponenten -> es gibt einen Fehler in meiner PerformRequest Methode, wobei mir aber auch das Zusammenspiel der ganzen enum, Moya Provider etc. noch nicht ganz klar sind.

Danke und bis später
Felix

On May 23, 2016, at 12:09 AM, Nils Fischer [email protected] wrote:

Der Code steht einfach in der Klasse, das macht keinen Sinn, denn da wird ja nie Code ausgeführt. In Klassen muss Code immer in einer Methode oder dergleichen stehen, die dann irgendwann ausgeführt werden kann. In den TODOs steht, welche Methoden implementiert werden müssen. Überleg dann mal, was da passiert, also versuch nachzuverfolgen, wie die Accessory Daten durch die App "fließen".


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub #8 (comment)

@FelixMei
Copy link
Author

Hey, nochmal eine Frage; ich hatte vorher versehentlich meine Änderungen im Pokebranch des geklonten Repositorys vorgenommen und habe diesen jetzt mit dem Master-Branch gemerged. Damit wollte ich jetzt einen Pull-Request machen - ich hoffe, dass das hat geklappt hat :)...

Danke

On May 30, 2016, at 11:56 AM, Felix Meissner [email protected] wrote:

Hey Nils,

danke für dein Feedback! ….Hast Du meine zwischenzeitlichen Verbesserungen angeschaut (das hab ich in einer Pull-Request aktualisiert..hoffe, da ist nichts schief gelaufen?) und könntest mir dazu noch ein Feedback/Support geben (Ich war am Montag deswegen nach der Vorlesung bei Dir :)…) ?

Zur aktuellen Aufgabe habe ich auch einen Pull-Request; mir fehlt noch ein bisschen der Überblick über die gesamten erforderlichen Komponenten -> es gibt einen Fehler in meiner PerformRequest Methode, wobei mir aber auch das Zusammenspiel der ganzen enum, Moya Provider etc. noch nicht ganz klar sind.

Danke und bis später
Felix

On May 23, 2016, at 12:09 AM, Nils Fischer <[email protected] mailto:[email protected]> wrote:

Der Code steht einfach in der Klasse, das macht keinen Sinn, denn da wird ja nie Code ausgeführt. In Klassen muss Code immer in einer Methode oder dergleichen stehen, die dann irgendwann ausgeführt werden kann. In den TODOs steht, welche Methoden implementiert werden müssen. Überleg dann mal, was da passiert, also versuch nachzuverfolgen, wie die Accessory Daten durch die App "fließen".


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub #8 (comment)

@nilsvu
Copy link
Member

nilsvu commented May 30, 2016

Hallo Felix, sorry hab nicht mehr dran gedacht.

  • Der Canvas View Controller sollte eine Content View haben, und darin jeweils eine Image View für das Foto und eine View mit transparentem Hintergrund für die Accessories:
    bildschirmfoto 2016-05-30 um 13 38 46
    Die Views kannst du dann mit Constraints an den Rand kleben und den Content Mode der Image View noch auf Aspect Fit setzen, sodass das Bild nicht verzerrt wird.
  • Die Buttons noch mit den IBActions verbinden
  • Die Accessory Cell und den Cancel Button mit der Exit Schaltfläche mit der Unwind Segue verbinden, nicht mit so einer Push Segue wie momentan:
    bildschirmfoto 2016-05-30 um 13 45 15

Dann klappts!

@FelixMei
Copy link
Author

FelixMei commented Jun 2, 2016

Hey Nils,

vielen Dank für deine Tips. Ich habe es jetzt dementsprechend überarbeitet, aber es stürzt noch immer ab... Habe einen neuen Pull-Request gemacht (mit Bezeichnung “updated”) - hat das funktioniert? (Ich habe generell noch nicht ganz durchschaut, wie ich mit der Seite “comparing changes” verfahren soll (?) …)

On May 30, 2016, at 2:06 PM, Nils Fischer [email protected] wrote:

Hallo Felix, sorry hab nicht mehr dran gedacht.

Der Canvas View Controller sollte eine Content View haben, und darin jeweils eine Image View für das Foto und eine View mit transparentem Hintergrund für die Accessories: https://cloud.githubusercontent.com/assets/746230/15648632/1524a808-266c-11e6-983a-6a846a013c22.png Die Views kannst du dann mit Constraints an den Rand kleben und den Content Mode der Image View noch auf Aspect Fit setzen, sodass das Bild nicht verzerrt wird.
Die Buttons noch mit den IBActions verbinden
Die Accessory Cell und den Cancel Button mit der Exit Schaltfläche mit der Unwind Segue verbinden, nicht mit so einer Push Segue wie momentan: https://cloud.githubusercontent.com/assets/746230/15648745/ebe76c90-266c-11e6-8d3f-c500016990e4.png Dann klappts!

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #8 (comment), or mute the thread https://github.com/notifications/unsubscribe/ARqxvjKr55zdfgN7O08M5kCX6uZCHN7Xks5qGtKvgaJpZM4IkFro.

@nilsvu
Copy link
Member

nilsvu commented Jun 5, 2016

Hi Felix, die Pull-Request wird mit jedem Commit, den du pusht, aktualisiert, das klappt also alles.

  • Die Views im CanvasViewController musst du noch mit Constraints an den Rand "kleben". Außerdem die eine transparent machen.
  • Die App stürzt ab, weil im Storyboard noch kein Identifier für die Unwind Segues vom AccessoryListViewController gesetzt sind

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants